Skip to content

Emoji Picker Fix - #1362

Open
PritamMishra065 wants to merge 2 commits into
RocketChat:developfrom
PritamMishra065:develop
Open

Emoji Picker Fix#1362
PritamMishra065 wants to merge 2 commits into
RocketChat:developfrom
PritamMishra065:develop

Conversation

@PritamMishra065

@PritamMishra065 PritamMishra065 commented Aug 21, 2026

Copy link
Copy Markdown

Brief Title

Keep Emoji Picker Open for Consecutive Emoji Selection

Acceptance Criteria fulfillment

  • Remove setEmojiOpen(false) from the handleEmojiClick callback in ChatInputFormattingToolbar.js.
  • Allow users to select multiple emojis consecutively without reopening the picker.
  • Ensure the picker still closes when clicking outside, toggling the emoji button, or sending the message.

Fixes #

Video/Screenshots

Screenshot 2026-08-21 121806

PR Test Details

  • Open EmbeddedChat and click the Emoji icon in the toolbar.
  • Select multiple emojis consecutively, such as 😀😅❤️.
  • Verify that the Emoji Picker remains open after each selection.
  • Click outside the picker and verify that it closes.
  • Toggle the emoji button and verify that the picker closes.
  • Send a message and verify that the picker closes as expected.

Note: The PR will be ready for live testing at https://rocketchat.github.io/EmbeddedChat/pulls/pr-#1362 after approval. Contributors are requested to replace #1362 with the actual PR number.

@CLAassistant

CLAassistant commented Aug 21, 2026

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ PritamMishra065
❌ Pritam Mishra


Pritam Mishra seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants